<html>
<head><meta charset="utf-8"><title>Criteria for triage to look at a PR · t-release/triage · Zulip Chat Archive</title></head>
<h2>Stream: <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/index.html">t-release/triage</a></h2>
<h3>Topic: <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html">Criteria for triage to look at a PR</a></h3>

<hr>

<base href="https://rust-lang.zulipchat.com">

<head><link href="https://rust-lang.github.io/zulip_archive/style.css" rel="stylesheet"></head>

<a name="210079174"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079174" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079174">(Sep 15 2020 at 00:31)</a>:</h4>
<p>what are the criteria then? even though it has a force-push it hasn't been reviewed in 16 days</p>



<a name="210079315"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079315" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079315">(Sep 15 2020 at 00:32)</a>:</h4>
<p>there was a review already by eddy (even though not the reviewer but "credible" enough - so the reviewer could be likely waiting on those changes to be made</p>



<a name="210079349"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079349" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079349">(Sep 15 2020 at 00:33)</a>:</h4>
<p>it says waiting-on-review though :/</p>



<a name="210079412"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079412" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079412">(Sep 15 2020 at 00:34)</a>:</h4>
<p>yes because it's still waiting on official reviewer, so it's a case of contradictory things being correct <span aria-label="grinning" class="emoji emoji-1f600" role="img" title="grinning">:grinning:</span></p>



<a name="210079428"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079428" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079428">(Sep 15 2020 at 00:34)</a>:</h4>
<p>(could have been s-waiting-on-author and wouldn't be wrong)</p>



<a name="210079496"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079496" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079496">(Sep 15 2020 at 00:35)</a>:</h4>
<p>well, if it had not had changes for 15 days, triage would have taken notice, right? I guess I'm wondering why rebasing to fix merge conflicts should reset the timer</p>



<a name="210079568"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079568" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079568">(Sep 15 2020 at 00:36)</a>:</h4>
<p>this wasn't a rebase though, it was changes to match a review, so slightly different</p>



<a name="210079619"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079619" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079619">(Sep 15 2020 at 00:37)</a>:</h4>
<p>no, that was to address a merge conflict: <a href="https://github.com/rust-lang/rust/pull/76043#issuecomment-686853500">https://github.com/rust-lang/rust/pull/76043#issuecomment-686853500</a><br>
you can't see the earlier changes to match a review because github doesn't show them after a force-push</p>



<a name="210079685"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079685" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079685">(Sep 15 2020 at 00:38)</a>:</h4>
<p>also generally reviewers wait for the author to resolve conflicts, so it makes sense to reset the timer</p>



<a name="210079699"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079699" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079699">(Sep 15 2020 at 00:38)</a>:</h4>
<p>it seems like that could lead to <em>really</em> long wait times</p>



<a name="210079711"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079711" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079711">(Sep 15 2020 at 00:39)</a>:</h4>
<p>imagine a PR that keeps hitting merge conflicts every 10 days</p>



<a name="210079803"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210079803" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210079803">(Sep 15 2020 at 00:40)</a>:</h4>
<p>that has happened tbh, and normally I would re-assign it to speed it once the 2nd iteration of conflicts have been resolved, but this is more from a manual inspection so has to be done on a case-by-case basis</p>



<a name="210080082"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080082" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080082">(Sep 15 2020 at 00:44)</a>:</h4>
<p>also the window was moved to 15 days recently, and was 7 days before that (started at 3 days before this group was formed), so chances of a pr constantly hitting conflicts without being picked in a triage window were slim</p>



<a name="210080122"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080122" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080122">(Sep 15 2020 at 00:44)</a>:</h4>
<p>what was the rationale for increasing that? 15 days without a review seems really frustrating :/</p>



<a name="210080132"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080132" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080132">(Sep 15 2020 at 00:44)</a>:</h4>
<p>it does but then reviewers need the bandwidth to review multiple prs</p>



<a name="210080232"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080232" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080232">(Sep 15 2020 at 00:46)</a>:</h4>
<p>and given that some people took a break from rust organisation or others got busy because of the changes due to the pandemic, it made sense to widen the window given the surge in prs reviewers have to look into</p>



<a name="210080245"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080245" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080245">(Sep 15 2020 at 00:46)</a>:</h4>
<p>yeah that's fair</p>



<a name="210080268"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210080268" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210080268">(Sep 15 2020 at 00:47)</a>:</h4>
<p>generally I try to reassign the smaller prs or review them if possible, to share the load and some other members do the same as well <span aria-label="smile" class="emoji emoji-1f642" role="img" title="smile">:smile:</span></p>



<a name="210081754"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210081754" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Charles Lew <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210081754">(Sep 15 2020 at 01:16)</a>:</h4>
<p><span class="user-mention" data-user-id="232545">@Joshua Nelson</span> I use <a href="https://github.com/rust-lang/rust/pulls?q=is%3Apr+is%3Aopen+-label%3AS-Blocked+sort%3Aupdated-asc+-is%3Adraft">this filter</a> you can see the conditions. And i clear up the &gt;15 days zone every friday.</p>



<a name="210117191"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210117191" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210117191">(Sep 15 2020 at 11:35)</a>:</h4>
<p>FWIW I don't personally find reviewing PRs with merge conflicts bad, but it depends on what those look like - normally they're just "git isn't smart" not actual conflicts, in which case a review doesn't hurt.</p>



<a name="210117318"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210117318" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Charles Lew <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210117318">(Sep 15 2020 at 11:36)</a>:</h4>
<p>Sometimes i wish i can <code>@rustbot rebase</code>, but i'm not sure how much is that possible.</p>



<a name="210117333"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210117333" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210117333">(Sep 15 2020 at 11:37)</a>:</h4>
<p><span class="user-mention" data-user-id="120823">@DPC</span> are we tracking anywhere something like reviewer latency? I know I try to check everything that's waiting on review for me every few days, but frequently PRs don't get put in my queue and are stuck in some other state</p>



<a name="210117368"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210117368" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> simulacrum <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210117368">(Sep 15 2020 at 11:37)</a>:</h4>
<p>Triagebot can't really help you rebase, it doesn't know how to resolve those conflicts</p>



<a name="210117739"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210117739" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Charles Lew <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210117739">(Sep 15 2020 at 11:42)</a>:</h4>
<p>Many times small merge conflicts can be resolved by <a href="https://www.git-scm.com/docs/git-mergetool"><code>mergetool</code></a>.</p>



<a name="210118297"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210118297" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Vadim Petrochenkov <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210118297">(Sep 15 2020 at 11:49)</a>:</h4>
<p><span class="user-mention silent" data-user-id="120823">DPC</span> <a href="#narrow/stream/242269-t-release.2Ftriage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR/near/210079685">said</a>:</p>
<blockquote>
<p>also generally reviewers wait for the author to resolve conflicts, so it makes sense to reset the timer</p>
</blockquote>
<p>That's not generally true, and we don't change the label to waiting-on-review on merge conflicts (<a href="https://github.com/rust-lang/rust-central-station/pull/34">https://github.com/rust-lang/rust-central-station/pull/34</a>).</p>



<a name="210121617"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210121617" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210121617">(Sep 15 2020 at 12:26)</a>:</h4>
<p>many reviews got delayed because the reviewer was waiting for conflicts to be resolved while the author was waiting for it to be reviewed to resolve the conflicts, thus resulting in a "deadlock". Also the reviewer has to most likely review it again after conflict resolution, so it makes sense to resolve it and then get reviewed and if possible approved straightaway.</p>



<a name="210123556"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210123556" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210123556">(Sep 15 2020 at 12:45)</a>:</h4>
<p>i know some conflicts are small and can be ignored but that involves someone checking them to ascertain if a conflict applies to a pr or not</p>



<a name="210123694"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210123694" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210123694">(Sep 15 2020 at 12:46)</a>:</h4>
<p>I think if we're having debate over it, it should be a case-by-case basis. So triage should assume the reviewer isn't waiting for a rebase, and the reviewer should comment if they are.</p>



<a name="210123887"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/210123887" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> DPC <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#210123887">(Sep 15 2020 at 12:48)</a>:</h4>
<p><span class="user-mention silent" data-user-id="232545">Joshua Nelson</span> <a href="#narrow/stream/242269-t-release.2Ftriage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR/near/210123694">said</a>:</p>
<blockquote>
<p>I think if we're having debate over it, it should be a case-by-case basis. So triage should assume the reviewer isn't waiting for a rebase, and the reviewer should comment if they are.</p>
</blockquote>
<p>that would depend on the type of conflict</p>



<a name="214353789"></a>
<h4><a href="https://rust-lang.zulipchat.com#narrow/stream/242269-t-release/triage/topic/Criteria%20for%20triage%20to%20look%20at%20a%20PR/near/214353789" class="zl"><img src="https://rust-lang.github.io/zulip_archive/assets/img/zulip.svg" alt="view this post on Zulip" style="width:20px;height:20px;"></a> Joshua Nelson <a href="https://rust-lang.github.io/zulip_archive/stream/242269-t-release/triage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR.html#214353789">(Oct 23 2020 at 19:07)</a>:</h4>
<p><span class="user-mention silent" data-user-id="120823">DPC</span> <a href="#narrow/stream/242269-t-release.2Ftriage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR/near/210123887">said</a>:</p>
<blockquote>
<p><span class="user-mention silent" data-user-id="232545">Joshua Nelson</span> <a href="#narrow/stream/242269-t-release.2Ftriage/topic/Criteria.20for.20triage.20to.20look.20at.20a.20PR/near/210123694">said</a>:</p>
<blockquote>
<p>I think if we're having debate over it, it should be a case-by-case basis. So triage should assume the reviewer isn't waiting for a rebase, and the reviewer should comment if they are.</p>
</blockquote>
<p>that would depend on the type of conflict</p>
</blockquote>
<p>that's kind of my point though - right now we do that for <em>all</em> conflicts, even minor ones that don't make it harder for the reviewer</p>



<hr><p>Last updated: Aug 07 2021 at 22:04 UTC</p>
</html>